Skip to content
This repository was archived by the owner on Dec 16, 2022. It is now read-only.

Sync libgit2-1.1.1 and main branches #17

Merged
merged 11 commits into from
Feb 9, 2022
Merged

Sync libgit2-1.1.1 and main branches #17

merged 11 commits into from
Feb 9, 2022

Conversation

pjbgf
Copy link
Member

@pjbgf pjbgf commented Feb 8, 2022

Sync branches to bring into the main the improvements recently added to branch libgit2-1.1.1:

  • Remove golang and XX tools from target image. The generated image now only contains the bare minimum files to consume the static libraries and their copyright notes.
  • The test image generates the target binaries through cross compilation, instead of emulated compilation at the target architectures. Using this approach should result in faster build times for SC/IAC in amd64/arm/v7.
  • Introduce make dev-test which allows for a check that the libraries can be used at development time at amd64 and arm64 linux targets.
  • Generate universal static libraries for MacOs to support both x86_64 and arm64 for MacOS 10.15 or newer.
  • Split macos release github workflow.

Paulo Gomes added 10 commits February 8, 2022 17:27
Signed-off-by: Paulo Gomes <[email protected]>
Aligns with what is being used for IAC and SC.

Signed-off-by: Paulo Gomes <[email protected]>
Allows local development without dependending on musl being installed on the development machine.

Signed-off-by: Paulo Gomes <[email protected]>
Signed-off-by: Paulo Gomes <[email protected]>
The changes on static.sh highlight the nuances of compiling for linux and mac, as well as cross compiling on the latter.

Signed-off-by: Paulo Gomes <[email protected]>
Signed-off-by: Paulo Gomes <[email protected]>
In linux no-stdio resulted in some intermittent issues whilst
using libgit2.

Signed-off-by: Paulo Gomes <[email protected]>
Signed-off-by: Paulo Gomes <[email protected]>
Signed-off-by: Paulo Gomes <[email protected]>
@hiddeco hiddeco merged commit 00d3e90 into fluxcd:main Feb 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants